revert: dirty handler pooled-only refresh (#119) - #121
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
按线上反馈回滚 dirty 相关行为改动(配套 juzibot/wechaty-puppet#111、juzibot/wechaty-puppet-service#100):contact 场景仍存在「dirty 两次才生效」现象,整体回退这一波 dirty 改动。
回滚 #119 后恢复旧行为:dirty 事件对 Contact/Room 无条件
find({id})+ready(true)(含未入池实例的主动拉取刷新)。#119 的池内限定恰好改变了 contact dirty 的 wechaty 层刷新语义,是 contact 残留症状的疑似成因之一。保留:#116(call 自循环防护 + alias 轮询,位于问题出现前的已知好版本 1.0.158 内,不属于此波)、#120 logger。
lint + tap 全量 33 spec files 通过。含 chore: 1.0.161 版本 commit。
🤖 Generated with Claude Code